chore(deps): bump basic-ftp to 5.3.0 and vite to 7.3.2 (replaces #42, #44)#46
Merged
Merged
Conversation
Supersedes #42 (npm-security group) and #44 (vite direct bump) — both blocked by branch-name governance. Re-created on a codex- compliant branch per docs/SECURITY.md. - basic-ftp: pnpm.overrides pin ^5.2.0 -> ^5.3.0 (security group) - vite: devDependency ^7.0.4 -> ^7.3.2 (minor, plugin-react 5 safe) #43 (picomatch 2.3.1 -> 2.3.2) intentionally skipped: picomatch@2.3.1 is a transitive dep pinned by a parent, and a minor-patch bump isn't worth an override hack. Picomatch@4.0.3 coexists at the latest major already. Validated locally: - pnpm install resolves cleanly to basic-ftp 5.3.0 and vite 7.3.2 - pnpm build: 636ms - pnpm test: 127/127 - pnpm typecheck: clean Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced Apr 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Two dep bumps on top of master. Skipping #43 (picomatch transitive) by design.
basic-ftp:pnpm.overridespin^5.2.0→^5.3.0(security group)vite: dev dep^7.0.4→^7.3.2(minor)Why
These Dependabot PRs were auto-created after the Wave 4 batch (#41 plugin-react) touched the lockfile. Both fail the branch-name governance guard; re-created on a codex-compliant branch per docs/SECURITY.md.
picomatch@4.0.3already coexists at the latest major.How
One commit. Edited
package.jsonfor the two direct/override entries, thenpnpm installto re-resolve. Final diff: 21/21 lines acrosspackage.json+pnpm-lock.yaml(only the two target packages' resolution lines).Testing
pnpm install,pnpm build,pnpm test,pnpm typecheckpnpm install→vite 7.3.1 → 7.3.2,basic-ftp 5.2.0 → 5.3.0(only these two deps moved)pnpm build→ 636ms, dist shape unchangedpnpm test→ 127/127pnpm typecheck→ cleanPerformance impact
Risk / Notes
vite 7.3.2is a patch bump over 7.3.1; plugin-react 5 is fully compatiblebasic-ftpis in the override set for deep-tree security tightening; consumers pull the newer version transparentlyScreenshots (UI only)
Lockfile rationale (if lockfile changed)
pnpm-lock.yamlupdated only for the two targeted packages; no other versions moved (verified via pnpm's install diff output).🤖 Generated with Claude Code